Skip to content

Test Savepoint rollback semantics - #584

Merged
SRombauts merged 1 commit into
masterfrom
savepoint-rollback-semantics
Sep 23, 2026
Merged

SRombauts merged 1 commit into
masterfrom
savepoint-rollback-semantics

Conversation

@SRombauts

@SRombauts SRombauts commented Sep 22, 2026

Copy link
Copy Markdown
Owner

Summary

  • add direct coverage for Savepoint::rollbackTo()
  • verify that changes made after rollbackTo() are committed only by release()
  • verify that scope exit still rolls back changes made after rollbackTo()
  • clarify the related test and destructor comments

No implementation or public API behavior changes.

@coveralls

coveralls commented Sep 22, 2026

Copy link
Copy Markdown

Coverage Status

coverage: 99.572% (-0.1%) from 99.715% — savepoint-rollback-semantics into master

@SRombauts SRombauts self-assigned this Sep 23, 2026
@SRombauts
SRombauts force-pushed the savepoint-rollback-semantics branch 2 times, most recently from e3e270e to 9a0e9b3 Compare September 23, 2026 09:11
@SRombauts SRombauts changed the title Cover Savepoint rollback-to scope-exit semantics Cover Savepoint rollback-to semantics Sep 23, 2026
@SRombauts
SRombauts force-pushed the savepoint-rollback-semantics branch from 9a0e9b3 to 8fa40dd Compare September 23, 2026 09:14
@SRombauts SRombauts changed the title Cover Savepoint rollback-to semantics Test Savepoint rollback semantics Sep 23, 2026
@SRombauts
SRombauts force-pushed the savepoint-rollback-semantics branch from 8fa40dd to b65deff Compare September 23, 2026 09:20
@SRombauts
SRombauts force-pushed the savepoint-rollback-semantics branch from b65deff to 7d30953 Compare September 23, 2026 09:30
@SRombauts
SRombauts merged commit da2c869 into master Sep 23, 2026
58 of 60 checks passed
@SRombauts
SRombauts deleted the savepoint-rollback-semantics branch September 23, 2026 09:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants